Skip to content

Add messages for motion primitives #228

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 3 commits into from
Jul 19, 2025

Conversation

urfeex
Copy link
Contributor

@urfeex urfeex commented Jul 7, 2025

These messages should reflect the motion primitives supported by most industrial robots.They are meant to being extended in the future in order to support a larger variety of motion implementations.

The motivation is to support executing robot motions as they are traditionally programmed: A series of primitives such as LIN, PTP, CIRC.

This PR should support ros-controls/ros2_controllers#1636

These messages should reflect the motion primitives supported by most industrial
robots.They are meant to being extended in the future in order to support
a larger variety of motion implementations.

The motivation is to support executing robot motions as they are
traditionally programmed: A series of primitives such as `LIN`,
`PTP`, `CIRC`.
mathias31415 added a commit to b-robotized-forks/ros2_controllers that referenced this pull request Jul 8, 2025
mathias31415 added a commit to b-robotized-forks/Universal_Robots_ROS2_Driver_MotionPrimitive that referenced this pull request Jul 8, 2025
bmagyar
bmagyar previously approved these changes Jul 17, 2025
Copy link
Member

@bmagyar bmagyar left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Great start, I only have that one question re. documenting a teeeeny bit more

Copy link
Contributor

@christophfroehlich christophfroehlich left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

Copy link
Member

@saikishor saikishor left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me

Co-authored-by: Sai Kishor Kothakota <saisastra3@gmail.com>
@christophfroehlich christophfroehlich merged commit 16742c2 into ros-controls:master Jul 19, 2025
10 checks passed
mergify bot pushed a commit that referenced this pull request Jul 19, 2025
saikishor pushed a commit that referenced this pull request Jul 19, 2025
(cherry picked from commit 16742c2)

Co-authored-by: Felix Exner <feex@universal-robots.com>
@urfeex urfeex deleted the motion_primitives branch July 21, 2025 08:08
mathias31415 added a commit to b-robotized-forks/Universal_Robots_ROS2_Driver_MotionPrimitive that referenced this pull request Jul 21, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants